Skip to content

Conversation

@mw-kapilg
Copy link
Contributor

@mw-kapilg mw-kapilg commented Oct 29, 2025

Summary

This PR includes:

  • Migration of run-command, run-build modules to this new common utilities repo
  • Test results summary view and logs grouping feature (explained in detail below)
  • Basic setup for this new TS repo

Details

This feature creates an in-platform summary for MATLAB test results if the user has a MATLAB Test license. Subsequent changes in other actions-repo will follow.

Key highlights:

  • Each row of the table represents the test file.
  • All tests for a single file are shown under that file.
  • The files with same name and different parent directories are shown separately and you can hover over the file name to see its path.
  • The table in table format allows easy management of test cases for a test file
  • You see the logs in-place for a failed/incomplete. If there are more than failures within a test case, say multiple verification failures, each verification failure will have its own dropdown and log box.
  • The help button redirects users to our plugin's configuration guide.

P.S. These utility functions would be utilized in run-command, run-tests and run-build actions to the generate test results summary

Screenshot 2025-09-05 145333

Some additional changes:

  • The test logs get grouped by test class now
  • Updated grouping of logs (similar to run-build task grouping)

Old:
Screenshot 2025-09-04 181035

New:
Screenshot 2025-09-04 181152

Screenshot 2025-09-04 180712

@mw-kapilg
Copy link
Contributor Author

I'm only able to add 1 reviewer in this pull request, does anyone know why that might be the case?

@sameagen-MW, can you review the pull request as well?

@sameagen-MW
Copy link

I'm only able to add 1 reviewer in this pull request, does anyone know why that might be the case?

@sameagen-MW, can you review the pull request as well?

Huh, that's interesting. I haven't seen that before and am not sure why that would be the case. Sorry I missed this message before, I'm happy to also review the change

Copy link

@sameagen-MW sameagen-MW left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, although I didn't look super closely at all of the files which were just copied over with no other changes.

I do notice that there's a message in the actions run which might indicate that the plugins aren't getting packaged correctly, here:
https://github.com/matlab-actions/common-utils/actions/runs/19126939215/job/54658903185?pr=1#step:4:34

Could you check up on that to make sure that it's not an issue? Other than that, looks ready to go!

Copy link

@davidbuzinski davidbuzinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm still not a fan of the integration testing style approach where we have to use test data to cover all branches and code paths. It also causes us to keep basically 3 copies of any test data file we make for each platform. The code changes look good though.

@davidbuzinski davidbuzinski self-requested a review November 6, 2025 20:35
@mw-kapilg mw-kapilg merged commit e6b2958 into main Dec 2, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants